Skip to content

p-interface: Change name for Pinocchio interface crate#73

Merged
joncinque merged 2 commits intosolana-program:mainfrom
joncinque:pinterface
Aug 4, 2025
Merged

p-interface: Change name for Pinocchio interface crate#73
joncinque merged 2 commits intosolana-program:mainfrom
joncinque:pinterface

Conversation

@joncinque
Copy link
Copy Markdown
Contributor

Problem

As part of https://github.com/orgs/anza-xyz/projects/27, we want to separate interfaces from programs for Agave.

The token repo already has spl-token-interface, but it's Pinocchio-only, which means it'll be currently difficult to start using in Agave.

Summary of changes

Change the crate name to pinocchio-token-interface and directory name to p-interface so that we can add interface in a future PR.

#### Problem

As part of https://github.com/orgs/anza-xyz/projects/27, we want to
separate interfaces from programs for Agave.

The token repo already has `spl-token-interface`, but it's
Pinocchio-only, which means it'll be currently difficult to start using
in Agave.

#### Summary of changes

Change the crate name to `pinocchio-token-interface` and directory name
to `p-interface` so that we can add `interface` in a future PR.
@joncinque
Copy link
Copy Markdown
Contributor Author

joncinque commented Jul 31, 2025

@rustopian once this lands, would you be able to create spl-token-interface which essentially contains everything needed by Agave? It would look similar to the PR for ATA, except with state.rs and native_mint.rs and error.rs on top of instruction.rs and lib.rs

Edit: and then after that, we publish spl-token-interface, and then update Agave to use that crate and eliminate all direct usage of spl-token

@joncinque
Copy link
Copy Markdown
Contributor Author

Sample PR in Agave for interface usage: anza-xyz/agave#7256

Copy link
Copy Markdown
Contributor

@rustopian rustopian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad missing this earlier today; looks great, all tests good.

@joncinque joncinque merged commit 8b7a81a into solana-program:main Aug 4, 2025
44 of 46 checks passed
@joncinque joncinque deleted the pinterface branch August 4, 2025 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants